-
Notifications
You must be signed in to change notification settings - Fork 54
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
✨ Add CRD Upgrade Safety Preflight check to the ClusterExtension controller #979
✨ Add CRD Upgrade Safety Preflight check to the ClusterExtension controller #979
Conversation
❌ Deploy Preview for olmv1 failed.
|
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #979 +/- ##
==========================================
- Coverage 79.40% 76.89% -2.51%
==========================================
Files 17 17
Lines 1170 1225 +55
==========================================
+ Hits 929 942 +13
- Misses 165 201 +36
- Partials 76 82 +6
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
de72c86
to
86634d8
Compare
af4db2b
to
da48d55
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes LGTM - thanks @rashmigottipati !
Would love to see a demo of this functioning in the next community call!
da48d55
to
d84277f
Compare
d84277f
to
7ac4221
Compare
…ller Signed-off-by: Rashmi Gottipati <[email protected]>
Signed-off-by: Rashmi Gottipati <[email protected]>
7ac4221
to
06914d2
Compare
cab41aa
…roller (operator-framework#979) * Add CRD Upgrade Safety Preflight check to the ClusterExtension controller Signed-off-by: Rashmi Gottipati <[email protected]> * add nil pointer check Signed-off-by: Rashmi Gottipati <[email protected]> --------- Signed-off-by: Rashmi Gottipati <[email protected]>
Description
ClusterExtension
controller to be configured with preflight checks and run them if they're enabledFixes #877
Reviewer Checklist